This repository has been archived by the owner on Aug 13, 2022. It is now read-only.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
#38 Feature/16
Feature/15
- 주석 빠진 부분 추가 - 로직만 작성했던 부분을 postman 테스트하면서 잘못 설정된 부분을 고쳤습니다.
푸시 적용할 곳이 아직 없네요.
added 2 commits
December 31, 2019 15:02
|
브랜치명은 왠만하면 한글을 넣지않고 이슈번호로 관리하시는게 좋습니다~ 뭔가 쓰더라도 영어로 쓰는게 좋고요 |
f-lab-dev
reviewed
Jan 2, 2020
| request.getItems(), request.getShopId()); | ||
|
|
||
| // 사장님에게 푸시 메세지 전송 | ||
| PushMessage pushMsg = new PushMessage("DelFood 주문", "새로운 주문이 들어왔습니다."); |
There was a problem hiding this comment.
객체 생성보다는 팩토리 메소드를 활용하면 더 깔끔해보일 것 같습니다~ 혹은 빌더를 사용하면 더 깔끔해보일 것 같네요.
그리고 이 경우는 객체의 값이 고정되어있으므로 상수로 선언하는게 좋을 것 같네요
| // 사장님에게 푸시 메세지 전송 | ||
| PushMessage pushMsg = new PushMessage("DelFood 주문", "새로운 주문이 들어왔습니다."); | ||
| String ownerId = shopService.getShop(request.getShopId()).getOwnerId(); | ||
| pushService.sendMessageToOwner(pushMsg, ownerId); |
There was a problem hiding this comment.
푸시메세지 전송은 비즈니스로직에 속합니다. Controller가 아닌 Service에 들어가야할 것 같습니다~
There was a problem hiding this comment.
그리고 Service에 들어가게되면 @Transactional을 타게 될텐데 푸시 전송에 실패해서 예외가 발생하면 주문을 롤백해야할까요? 이 점에 대해서도 고민해보시면 좋을 것 같습니다~
- 주문 승인 시 도착예정시간 설정 가능 - 주문 승인 시 고객에게 알림 메세지 전송
- 회원 가입 - 로그인 - 로그아웃 - 아이디와 비밀번호로 유효성 검사 - 비밀번호 변경 - 메일 변경
[#46] 주문시 쿠폰 적용
Coupon에 대한 테스트코드 작성
f-lab-dev
reviewed
Jan 14, 2020
|
|
||
| @AllArgsConstructor | ||
| public static enum Type { | ||
| addOrderRequest(new PushMessage("DelFood 주문", "새로운 주문이 들어왔습니다")), |
There was a problem hiding this comment.
이곳은 enum보다는 PushMessage 안에 상수로 정의하는게 사용성면에서 더 좋을 것 같네요~
[#67] 라이더 회원 정보 서비스
Order coupon
f-lab-dev
approved these changes
Jan 16, 2020
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
푸시 적용할 곳이 아직 없네요.